Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix static link issue in tooling #16174

Merged
merged 28 commits into from
Jan 31, 2025
Merged

Fix static link issue in tooling #16174

merged 28 commits into from
Jan 31, 2025

Conversation

AnieeG
Copy link
Contributor

@AnieeG AnieeG commented Jan 31, 2025

Requires

Supports

AnieeG added 24 commits December 6, 2024 16:47
@AnieeG AnieeG requested review from kylesmartin and a team as code owners January 31, 2025 19:19
@AnieeG AnieeG requested a review from winder January 31, 2025 19:19
@@ -56,6 +58,40 @@ func DeployLinkToken(e deployment.Environment, chains []uint64) (deployment.Chan
return deployment.ChangesetOutput{AddressBook: newAddresses}, nil
}

// DeployStaticLinkToken deploys a static link token contract to the chain identified by the ChainSelector.
func DeployStaticLinkToken(e deployment.Environment, chains []uint64) (deployment.ChangesetOutput, error) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be unexported? Function sig does not exactly match that of a changeset.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nvm, thrown off by primitive type as "config"

kylesmartin
kylesmartin previously approved these changes Jan 31, 2025
Copy link
Contributor

github-actions bot commented Jan 31, 2025

AER Report: CI Core ran successfully ✅

aer_workflow , commit

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

tt-cll
tt-cll previously approved these changes Jan 31, 2025
Copy link
Contributor

@tt-cll tt-cll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hoping we can split out everything so it's less brittle soon

@AnieeG AnieeG dismissed stale reviews from tt-cll and kylesmartin via 6d82409 January 31, 2025 20:23
@AnieeG AnieeG enabled auto-merge January 31, 2025 22:00
@AnieeG AnieeG added this pull request to the merge queue Jan 31, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 31, 2025
@AnieeG AnieeG added this pull request to the merge queue Jan 31, 2025
Merged via the queue into develop with commit 91bc947 Jan 31, 2025
204 of 206 checks passed
@AnieeG AnieeG deleted the fix-static-link branch January 31, 2025 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants